New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Solution #802

Open

YerepSemyon wants to merge 2 commits into mate-academy:master from YerepSemyon:develop

YerepSemyon commented Sep 13, 2024


          Solution

e5b3229

vsmutok suggested changes

View reviewed changes

templates/includes/sidebar.html Outdated

Comment on lines 2 to 15

+                {% if user.is_authenticated %}
+              		<li>
+                    User: <a href="{% url "taxi:driver-detail" pk=user.id %}">{{ user.username }}</a>
+              		</li>
+              		<li>
+              			<a href="{% url 'logout' %}">Logout</a>
+              		</li>
+              	{% else %}
+              		<li>
+              			<a href="{% url 'login' %}">Login</a>
+              		</li>
+              	{% endif %}
+                <hr>
+              	<br>

vsmutok Sep 13, 2024

please fix html format

Author

YerepSemyon Sep 14, 2024 •

edited

Loading

interesting, I don't know why it is displayed like that here, because everything looks fine in PyCharm. I will write this part from scratch


          fixed format in sidebar.html

9439ba4

YerepSemyon requested a review from vsmutok

September 14, 2024 12:10

vsmutok approved these changes

View reviewed changes

vsmutok left a comment

Looks Good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet